Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[macOS] Fix curl error "argument list too long" #397

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

HLFH
Copy link

@HLFH HLFH commented May 4, 2019

See #396 for details.
The requestbody.txt file is being saved to /Users/MYSUSER/Library/Containers/com.microsoft.Excel/Data and will be removed once the curl command has been executed.

See VBA-tools#396 for details.
The `requestbody.txt` file is being saved to `/Users/MYSUSER/Library/Containers/com.microsoft.Excel/Data` and will be removed once the `curl` command has been executed.
@mohawk2
Copy link

mohawk2 commented May 28, 2020

Surely the file should be placed in a temporary directory instead? What if the local directory is read-only?

@michaelterryio
Copy link

Thanks @HLFH!

@fzumstein
Copy link

Surely the file should be placed in a temporary directory instead? What if the local directory is read-only?

@mohawk2 /Users/MYSUSER/Library/Containers/com.microsoft.Excel/Data is indeed one of the few writable directories with modern masOS Excel (since 2016/365): since Excel is sandboxed, external directories (such as /tmp) would require approval first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants